home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part2 / 11690 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  1.9 KB
  1. Path: mac007016.shef.ac.uk!user
  2. From: m.b.greenwood@sheffield.ac.uk (Mike Greenwood)
  3. Newsgroups: comp.lang.c
  4. Subject: Re: Please help me
  5. Date: Tue, 26 Mar 1996 11:47:33 +0000
  6. Organization: University of Sheffield
  7. Message-ID: <m.b.greenwood-2603961147330001@mac007016.shef.ac.uk>
  8. References: <4j6nrl$lfk@badger.wmin.ac.uk>
  9. NNTP-Posting-Host: mac007016.shef.ac.uk
  10.  
  11. In article <4j6nrl$lfk@badger.wmin.ac.uk>, darec@westminster.ac.uk
  12. (Nadarajah Thavaneethan) wrote:
  13.  
  14. > #define  SIZE 60
  15.  
  16. > void main(void)
  17. > {
  18. > 
  19. >         char c[SIZE];
  20. > int index, next;
  21. > 
  22. etc
  23. >                compchar(c[next-1],c[next]);
  24.  
  25. >             printf("\n the line is now %s\n",c);
  26. >  }
  27. > 
  28. >  void compchar (char c1, char c2)
  29. >  {
  30. > 
  31. >  char p;
  32. > 
  33. > 
  34. >           if (c1 > c2)
  35. >               p = c1;
  36. >               c1 = c2;
  37. >               c2 = p;
  38. >  }
  39.  
  40. > This program reads a line of data and sorts it into ascending ASCII sequence
  41. >  it doesn't but it should.
  42.  
  43. Hi there,
  44.  
  45. I think this is quite a simple problem. When you call the function
  46. compchar, and swap if necessary, only a copy of c1 and c2 is sent to the
  47. function. This means that c[next-1] and c[next] in the main() function
  48. don't get swapped even if c1 and c2 do. One way to avoid this is to
  49. declare c[SIZE] as global, i.e. before main(), and call 'void
  50. compchar(void)'.
  51. Inside the compchar() function you could then operate with:
  52.    if (c[next-1] > c[next])
  53.       etc...
  54.  
  55. Another way around this is to use pointers, because pointers point to the
  56. real values whatever, and a copy of a pointer still points to the real
  57. variable. This is the better way, but requires a heavier modificaton to
  58. your program.
  59.  
  60. By the way, if you want to avoid nasty responses, declare:
  61.    int main()
  62.    ...
  63.    return (0);
  64.  
  65. rather than 
  66.    void main(void)
  67.  
  68. the latter is not standard despite what certain books may say. I used to
  69. do it but have seen the error of my ways thanks to this newsgroup.
  70.  
  71. Hope this helps.
  72.  
  73. Cheers
  74. Mike Greenwood
  75.